View GitHub Repository simhelpdesk

Schedule Inspection
Subscribe

time.js ➔ startTime   A
last analyzed

↳ Parent: Project

Complexity

Conditions 1
Paths 1

Size

Total Lines 19
Code Lines 18

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 18
nc 1
nop 0
dl 0
loc 19
rs 9.5
c 1
b 0
f 0

1 Function

Rating   Name   Duplication   Size   Complexity  
A time.js ➔ ... ➔ setTimeout 0 1 1
1 
function startTime()
2 
{ 	var today=new Date();
3 
	var weekday=new Array(7);
0 ignored issues
– show
Unused Code introduced by
The assignment to variable weekday seems to be never used. Consider removing it.
Loading history...
4 
	var weekday=["Minggu","Senin","Selasa","Rabu","Kamis","Jumat","Sabtu"];
0 ignored issues
– show
Comprehensibility Naming Best Practice introduced by
The variable weekday already seems to be declared on line 3. Consider using another variable name or omitting the var keyword.

This check looks for variables that are declared in multiple lines. There may be several reasons for this.

In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs.

If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared.

Loading history...
5 
	var monthname=new Array(12);
0 ignored issues
– show
Unused Code introduced by
The assignment to variable monthname seems to be never used. Consider removing it.
Loading history...
6 
	var monthname=["Jan","Feb","Mar","Apr","May","Jun","Jul","Aug","Sep","Oct","Nov","Dec"];
0 ignored issues
– show
Comprehensibility Naming Best Practice introduced by
The variable monthname already seems to be declared on line 5. Consider using another variable name or omitting the var keyword.

This check looks for variables that are declared in multiple lines. There may be several reasons for this.

In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs.

If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared.

Loading history...
7 
	var dayname=weekday[today.getDay()];
8 
	var day=today.getDate();
9 
	var month=monthname[today.getMonth()];
10 
	var year=today.getFullYear();
11 
	var h=today.getHours();
12 
	var m=today.getMinutes();
13 
	var s=today.getSeconds();
14 
	h=checkTime(h);
15 
	m=checkTime(m);
16 
	s=checkTime(s);
17 
	document.getElementById('clocktime').innerHTML=dayname+", "+day+"-"+month+"-"+year+", "+h+":"+m+":"+s;
18 
	t=setTimeout(function(){startTime()},500);
0 ignored issues
– show
Bug introduced by
The variable t seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.t.
Loading history...
19 
}
20 
// function checkTime to add a zero in front of numbers<10
21 
function checkTime(i)
22 
{	if(i<10){i="0"+i;}
23 
	return i;
24 
}